src/app/shared/lib/getColor.ts (2)

1-5: 의도는 색상 ‘인덱스’ 반환이므로 함수명 혼동 가능
getColor 라는 이름은 문자열 hex 값을 돌려줄 것처럼 보이지만 실제로는 배열 인덱스를 반환합니다.

-export function getColor(text: string, customColors: string[]): number {
+export function getColorIndex(text: string, palette: string[]): number {

이후 호출부에서 palette[getColorIndex(tag, palette)] 형태로 쓰면 의미가 더 명확해집니다.

---

1-5: 단순 합 해시 → 아나그램 충돌 위험
abc 와 cba 가 동일 인덱스를 갖게 됩니다. UX 상 큰 문제는 아니지만, 충돌을 줄이고 싶다면 DJB2 같은 경량 해시를 쓰거나 hash = ((acc << 5) - acc) + charCode 패턴을 고려해 보세요.

.github/workflows/ci.yml (1)

11-12: self-hosted 러너 사용 시 실패 대체 경로 필요
사내 러너 장애 시 워크플로 전체가 멈춥니다.

runs-on: [self-hosted, ubuntu-latest]

처럼 fallback 러너를 지정하거나, runner-group을 활용해 가용성을 확보해 주세요.

src/app/globals.css (1)

69-71: 드래그 Hover 색상에 다크 모드 변형을 함께 정의해 주세요.

.BG-drag-hovered 클래스는 라이트 모드 전용 색상(bg-blue-100)만 적용돼 있어 다크 모드에서 시각 피드백이 사라집니다. 다른 유틸리티처럼 dark:bg-[#2C3641] 등을 추가해 두 테마 모두 일관된 UX를 보장해 주세요.

src/app/shared/components/common/header/Collaborator/Tooltip.tsx (3)

20-30: 스크롤/리사이즈 시 위치 재계산 누락

툴팁은 visible 발생 시 한 번만 좌표를 계산합니다. 창 크기 변경, 스크롤 이동 후에도 툴팁이 그대로 남아 위치가 어긋날 수 있으므로 window.scroll, resize 이벤트에 대응해 좌표를 갱신하거나 IntersectionObserver/ResizeObserver를 고려해 주세요.

---

40-48: 다크 모드 색상 하드코딩

배경·글자색을 직접 HEX로 지정해 다크 모드 전환 시 일관성을 잃습니다. Tailwind 클래스(BG-white, Text-black 등)나 CSS 변수 사용으로 테마 의존성을 제거하는 편이 유지보수에 유리합니다.

---

60-68: 불필요한 래퍼 제거로 DOM 최적화 가능

<div className="relative flex items-center">는 자식이 하나뿐인 경우가 대부분이므로, 필요 없으면 Fragment로 대체해 DOM 깊이를 줄일 수 있습니다.

src/app/(auth)/login/page.tsx (1)

5-10: 불필요한 Fragment 제거

단일 자식만 반환하므로 빈 Fragment(<> </>)가 필요 없습니다. 간단히 <LoginForm />를 반환하도록 수정해 가독성을 높여 주세요.

-  return (
-    <>
-      <LoginForm />
-    </>
-  )
+  return <LoginForm />

src/app/shared/lib/cn.ts (1)

4-6: clsx 인수 전달 방식 개선 제안

clsx 는 가변 인자를 받도록 설계되어 있어 clsx(...inputs) 로 전달하는 편이 타입 추론과 가독성 모두에서 일반적입니다. 현재처럼 배열 하나를 넘겨도 동작은 하지만 타입 경고가 발생하거나 중첩 배열이 만들어질 수 있습니다.

-export function cn(...inputs: ClassValue[]) {
-  return twMerge(clsx(inputs))
+export function cn(...inputs: ClassValue[]) {
+  return twMerge(clsx(...inputs))
 }

src/app/shared/components/common/UserInfo.tsx (1)

11-18: 접근성 & 재사용성을 위한 forwardRef 적용 고려

외부에서 UserInfo 요소에 직접 포커스·스크롤 제어가 필요한 경우가 잦습니다(예: 메뉴 트리거). forwardRef 를 적용해 DOM 참조를 노출하면 컴포넌트 활용도가 높아집니다. 한편, span 대신 p/div 등 블록 요소가 필요할 때도 유연하게 대응할 수 있습니다.
간단히 아래와 같이 개선 가능합니다.

-import { Avatar } from './Avatar'
+import { forwardRef } from 'react'
+import { Avatar } from './Avatar'
 ...
-export function UserInfo({ nickname, imageUrl, size = 36 }: UserInfoProps) {
-  return (
+export const UserInfo = forwardRef<HTMLDivElement, UserInfoProps>(
+  ({ nickname, imageUrl, size = 36 }, ref) => (
     <div ref={ref} className="flex items-center gap-4">
       <Avatar nickname={nickname} imageUrl={imageUrl} size={size} />
       <span className="text-sm font-semibold">{nickname}</span>
     </div>
-  )
-}
+  ),
+)

src/app/shared/components/common/header/UserDropdown.tsx (1)

20-28: width="w-6" 값이 Dropdown 내부 스타일과 맞지 않을 가능성

w-6(24 px) 은 메뉴 아이템(텍스트 포함)에 비해 지나치게 좁아 클릭 영역이 잘리거나 overflow 가 발생할 수 있습니다. 사용 의도를 재확인하고 w-48 등 충분한 가로폭을 지정해 주세요.

src/app/features/auth/schemas/loginValidation.ts (1)

1-16: 패스워드 복잡도 규칙 추가 고려

현행 검증은 길이만 확인합니다. 서비스 정책에 따라 대·소문자, 숫자, 특수문자 포함 여부 등을 추가하면 보안 수준을 높일 수 있습니다. 필요 시 정규식 패턴과 맞춤 메시지를 포함해 주세요.

src/app/dashboard/[id]/Card/Tags.tsx (1)

6-7: 컬러 배열은 컴포넌트 외부로 이동해 재-렌더시 재생성을 방지하세요
bgColors와 textColors를 함수 내부에 선언하면 렌더링마다 새 배열이 생성됩니다. 불필요한 메모리 할당을 줄이려면 파일 상단(모듈 스코프)으로 옮기는 편이 좋습니다.

-  const bgColors = ['#F9EEE3', '#E7F7DB', '#F7DBF0', '#DBE6F7']
-  const textColors = ['#D58D49', '#86D549', '#D549B6', '#4981D5']
+const BG_COLORS = ['#F9EEE3', '#E7F7DB', '#F7DBF0', '#DBE6F7'] as const
+const TEXT_COLORS = ['#D58D49', '#86D549', '#D549B6', '#4981D5'] as const
+
+export default function Tags({ tags }: { tags: string[] }) {

src/app/features/auth/hooks/useLoginSubmit.ts (1)

17-24: 에러 메시지 타입이 불확실할 때 안전한 디폴트 추가 권장
Axios 응답 message가 문자열이 아닐 가능성을 대비해 typeof message === 'string' 체크 후 사용하면 토스트 출력 오류를 방지할 수 있습니다.

-        const message = e.response?.data?.message
-        toast.error(message ?? '로그인 실패')
+        const message = e.response?.data?.message
+        toast.error(
+          typeof message === 'string' && message.length > 0
+            ? message
+            : '로그인 실패',
+        )

src/app/dashboard/[id]/store/useDragStore.ts (2)

4-7: 타입 명명 규칙 통일 필요

interface draggingCard 와 같이 소문자로 시작하는 타입은 팀 컨벤션(PascalCase)을 벗어납니다. DraggingCard 로 수정하거나 아예 별도 타입 없이 Card 자체를 직접 사용해도 충분합니다.

-interface draggingCard {
+interface DraggingCard {
   cardData: Card
 }

---

10-11: 파라미터 타입 중복 정의

setDraggingCard 의 인자 타입을 { cardData: Card } 로 직접 지정하기보다는 바로 위에서 정의한 DraggingCard(또는 동일 구조)를 재사용하면 타입 중복을 줄일 수 있습니다.

- setDraggingCard: (data: { cardData: Card }) => void
+ setDraggingCard: (data: DraggingCard) => void

src/app/features/auth/components/LoginForm.tsx (2)

41-44: autoComplete 속성 값 재검토

비밀번호 입력의 autoComplete="off" 는 일부 브라우저에서 무시되며 UX 를 떨어뜨릴 수 있습니다. 일반적으로 로그인 폼에는 autoComplete="current-password" 를 권장합니다.

- autoComplete="off"
+ autoComplete="current-password"

---

51-55: 비활성 상태 스타일 Tailwind 내장 variant 활용

disabled: variant를 사용하면 조건부 클래스를 줄일 수 있습니다.

- isValid && !isSubmitting ? 'BG-blue' : 'BG-blue-disabled',
+ 'BG-blue disabled:BG-blue-disabled',

이를 통해 로직 복잡도를 낮추고 스타일 일관성을 유지할 수 있습니다.

src/app/(auth)/signin/page.tsx (1)

57-57: 임시 입력 필드 및 페이지 정리 필요

<Input labelName="안녕" name="text" /> 는 실제 로그인·회원가입 흐름과 무관한 임시 코드로 보입니다. 기능 구현이 끝났다면 제거하거나 스토리북/샘플 컴포넌트로 분리해 주세요.

src/app/shared/components/common/sidebar/Sidebar.tsx (2)

14-61: 목데이터 정의 위치 및 재생성 비용

mockDashboards 가 컴포넌트 내부에 있어 렌더링마다 새 배열이 생성됩니다. 실제 API 연동 전이라도 상수는 컴포넌트 외부로 이동하거나 useMemo 로 래핑해 불필요한 재렌더링을 방지하는 편이 좋습니다.

---

63-66: 라우팅 시 에러 방지

router.push 호출 시 현재 경로와 동일하면 불필요한 네비게이션이 발생합니다. 클릭한 대시보드가 이미 활성화되어 있을 때는 early-return 하도록 검토해 주세요.

if (pathname !== `/dashboard/${dashboardId}`) {
  router.push(`/dashboard/${dashboardId}`)
}

src/app/shared/components/common/sidebar/DashboardItem.tsx (1)

36-43: 이미지 로딩 상태 처리 고려

왕관 이미지에 로딩 상태나 에러 처리를 추가하면 사용자 경험이 향상될 수 있습니다.

src/app/dashboard/[id]/page.tsx (1)

32-32: 사용하지 않는 주석 코드 제거

주석 처리된 코드는 제거하는 것이 좋습니다.

-    // setDraggingCard({ cardData: cardData }) // 전역상태에, 현재 드래그할 카드 저장(후에 뮤테이션 함수에 전달해서 캐시 업데이트에 사용)

src/app/shared/types/dashboard.ts (1)

2-10: 날짜 필드에 string 대신 Date 사용 고려
createdAt, updatedAt 필드를 문자열로 두면 소비 측에서 매번 파싱 로직을 가져가야 합니다. API 스펙을 조정할 수 있다면 Date 타입(ISO-8601 문자열을 받아서 즉시 파싱)으로 정의하거나, 최소한 별도의 type Timestamp = string alias로 명시해 두면 의도가 더 분명해집니다.

src/app/shared/components/common/Avatar.tsx (1)

13-23: customColors 분리 제안
customColors 상수가 컴포넌트 내부에 정의돼 있어, 다른 곳에서 같은 팔레트를 써야 할 때 중복 선언될 가능성이 있습니다. shared/constants/colors.ts 같은 파일로 이동해 재사용성을 높여 주세요.

src/app/dashboard/[id]/Card/Card.tsx (2)

20-24: HTML data-attribute에 대용량 JSON 직렬화 주의
data-card-data에 JSON.stringify(card)를 그대로 넣으면 DOM 노드에 불필요하게 큰 문자열이 삽입됩니다. DnD 로직에서 꼭 필요하다면 id만 두고, 상세 데이터는 Zustand나 context에서 조회하는 쪽이 메모리·성능 모두 유리합니다.

---

20-23: draggable 속성은 boolean 리터럴 사용
React에서는 문자열 "true" 대신 draggable={true}를 권장합니다.

src/app/shared/components/common/header/Collaborator/CollaboratorList.tsx (1)

49-56: “+N” 툴팁 내용 부정확 가능성
visibleCollaborators[0].nickname 은 이미 화면에 표시된 사람입니다. 초과 인원을 설명하려면 collaborators[MAX_VISIBLE].nickname 등, 실제로 숨겨진 첫 번째 사용자를 참조하는 편이 직관적입니다.

src/app/shared/components/Input.tsx (1)

41-68: 접근성 속성 추가
에러 메시지가 있을 때 aria-describedby를 입력 요소에 연결하면 스크린리더가 오류를 읽어줄 수 있습니다. 또한 비밀번호 토글 버튼에 aria-label을 제공해 주세요.

-  {...rest}
+  aria-describedby={hasError ? `${name}-error` : undefined}
+  {...rest}
 ...
-  alt={showPassword ? '비밀번호 보기' : '비밀번호 숨기기'}
+  alt=""
+  aria-label={showPassword ? '비밀번호 보기' : '비밀번호 숨기기'}

src/app/dashboard/[id]/Column/Column.tsx (1)

1-1: 사용되지 않는 useQueryClient import

이 파일에서는 useQueryClient가 참조되지 않습니다. 불필요한 import는 제거하여 번들 크기와 린트 경고를 줄이는 것이 좋습니다.

src/app/shared/components/common/Dropdown/Dropdown.tsx (2)

30-40: getWidthValue가 두 가지만 지원해 확장성이 떨어집니다

지금은 w-5, w-6만 하드코딩되어 있어 새로운 폭이 필요할 때마다 코드를 수정해야 합니다.
Tailwind theme('spacing')을 활용하거나 전달된 클래스에서 숫자를 추출해 rem으로 변환하도록 일반화하면 유지보수가 쉬워집니다.

---

126-133: 접근성 속성 추가 제안

트리거 요소에 aria-haspopup="menu"와 aria-expanded={open}를 부여하면 스크린리더가 드롭다운 상태를 올바르게 알릴 수 있습니다.

- <div
+ <div
+   aria-haspopup="menu"
+   aria-expanded={open}
   ref={triggerRef}

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro